Skip to content

feat: add GitHub Actions integration test workflow with full cross-process e2e coverage#11

Closed
Copilot wants to merge 4 commits into
mainfrom
copilot/add-full-integration-test-workflow
Closed

feat: add GitHub Actions integration test workflow with full cross-process e2e coverage#11
Copilot wants to merge 4 commits into
mainfrom
copilot/add-full-integration-test-workflow

Conversation

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown
  • Investigate CI failures
  • Root-cause analysis
  • Fix 1 – conftest.py: replace subprocess.PIPE pair with stdout=DEVNULL + stderr-watching daemon thread; add -u for unbuffered output; detect readiness from "Listening for tasks" log line; wait up to 120 s (supports sandbox boot)
  • Fix 2 – conftest.py: set amqp://localhost?heartbeat=600 to prevent AMQP idle-disconnect
  • Fix 3 – integration.yml: use rabbitmq:3-management-alpine (matches unit-tests job)
  • Verified locally: local+signing=false+sandbox=false → 8 passed, 6 skipped; local+signing=true+sandbox=false → 11 passed, 3 skipped
  • Code review feedback addressed: removed unnecessary broad except Exception in watcher thread

Copilot AI and others added 2 commits April 21, 2026 19:41
Copilot AI and others added 2 commits April 21, 2026 21:20
…ckends

- conftest.py: add `-u` flag for unbuffered Python output in worker subprocess
- conftest.py: replace subprocess.PIPE pair with stdout=DEVNULL + stderr-watching
  background thread; this prevents pipe-buffer deadlocks and detects readiness
  from the "Listening for tasks" log line (works for all backends incl. sandbox)
- conftest.py: wait up to 120 s for worker readiness (instead of 4 s sleep);
  sandbox container boot can take ~60 s on a cold CI runner
- conftest.py: amqp URL now includes ?heartbeat=600 to prevent AMQP
  idle-disconnect; without this the default 60 s heartbeat causes the
  listen() generator to exit and stops the worker consuming any tasks
- integration.yml: use rabbitmq:3-management-alpine (matches unit-tests job)

Agent-Logs-Url: https://github.com/codeSamuraii/pyfuse/sessions/3fc206b9-a7f6-45e5-9770-7a59c2f59f3c

Co-authored-by: codeSamuraii <17270548+codeSamuraii@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants